fix(linter): panic in consistent-function-scoping #5613
Conversation
Your org has enabled the Graphite merge queue for merging into mainAdd the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
@DonIsaac why was the symbol id |
|
No idea. If there's a bug in semantic, it still needs fixing. Either way, the linter should not panic because of bugs in semantic. |
CodSpeed Performance ReportMerging #5613 will not alter performanceComparing Summary
|
agreed. although we have a lot of |
|
maybe we should add debug asserts here for the |
|
This is they only case that makes it none, let me create an separate issue. |
Merge activity
|
Boshen
force-pushed
the
don/09-08-fix_linter_panic_in_consistent-function-scoping
branch
from
e1acafc to
af6d240
Compare
graphite-app
bot
deleted the
don/09-08-fix_linter_panic_in_consistent-function-scoping
branch
## [0.9.4] - 2024-09-12 ### Features - 9ca2593 linter: Eslint/sort-keys (#4845) (Na'aman Hirschfeld) - 023c160 linter: Impl `Serialize` for `OxlintConfig` (#5594) (DonIsaac) - 24d6a47 linter: Implement `eslint/no-invalid-regexp` (#5443) (Boshen) ### Bug Fixes - af6d240 linter: Panic in consistent-function-scoping (#5613) (DonIsaac) - 54e2e76 linter: `react/no_set_state` + `react/no_string_refs` rules find correct parent (#5615) (overlookmotel) - 3b87ac4 linter: Fix no_unused_vars panic when encountering unicode (#5582) (Boshen) ### Performance - bfe9186 linter: Use `cow_replace` instead of `replace` (#5643) (dalaoshu) - e3ae5db linter: Use cow_to_ascii_lowercase/uppercase (#5637) (heygsc) - a0370bf linter: Use cow_utils in no_script_url (#5633) (heygsc) - 37e922c linter: `eslint/no_shadow_restricted_names` use `run_on_symbol` (#5618) (overlookmotel) - 0b7fccf linter: `react/no_set_state` + `react/no_string_refs` rules reduce iteration over ancestors (#5616) (overlookmotel) - 2c3f3fe linter: Make `jsx_key` slightly faster (#5585) (Boshen) - cd81d12 linter: Add `should_run` to check path only once to nextjs/no_typos (#5584) (Boshen) - d18c896 rust: Use `cow_utils` instead (#5664) (dalaoshu) ### Documentation - 64f9575 linter: Add plugin usage to example with configuration (Boshen) - 8c9179d linter: Fix typos (#5591) (Brian Donovan) ### Refactor - 9e9435f linter: Add `LintFilter` (#5685) (DonIsaac) - 4f70fe5 linter: Start internal/external split of LintPluginOptions (#5660) (DonIsaac) - 5ae9b48 linter: Start internal/external split of `OxlintOptions` (#5659) (DonIsaac) - c8bc6f0 linter: Use `std::ptr::eq` (#5649) (overlookmotel) - a37c064 linter: Use `ContentHash` for `no_duplicate_case`; remove `calculate_hash` (#5648) (Boshen) - 0b3c1d7 linter: Start internal/external split of `OxlintConfig` (#5595) (DonIsaac) - 89bdf55 linter: Inline `Rule` trait default methods (#5619) (overlookmotel) - afea8d5 linter: Rename `Rule` trait method params (#5617) (overlookmotel) - 4e748b5 linter: Replace ast "compare by hash" to "compare by content" (#5602) (dalaoshu) - bac03e3 linter: Make fields of `LintServiceOptions` private (#5593) (DonIsaac) - 2661d8b linter: Jest prefer_strict_equal (#5588) (IWANABETHATGUY) - 20d0068 oxlint: Move cli-related exports to `cli` module (#5139) (DonIsaac) - 067f9b5 semantic: Introduce `IsGlobalReference` trait (#5672) (Boshen)- 26d9235 Enable clippy::ref_as_ptr (#5577) (夕舞八弦) ### Testing - 8e79f8d linter: Add class method test cases for `oxc/no-async-await` (#5550) (DonIsaac) - 3835189 linter: Add test case for no_unused_vars in 3b87ac4 (Boshen) - 5f27551 linter: Add a passing case to no_undef (#5580) (Boshen) --------- Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Closes #5608